[CRM-14077] Lift PHP cap to <8.4 and add 8.3 to CI#5
Conversation
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Greptile SummaryThis PR lifts the PHP version ceiling in
Confidence Score: 5/5Safe to merge — the two-line change correctly narrows the ceiling to Both changed files are configuration only. The composer constraint and CI matrix are perfectly aligned: every PHP version the package now accepts (7.4 through 8.3) is exercised in CI with both No files require special attention.
|
| Filename | Overview |
|---|---|
| composer.json | PHP constraint lifted from >=7.4 <8.3 to >=7.4 <8.4; aligns with CI coverage (8.3 is the highest tested version). |
| .github/workflows/pr_test.yml | PHP 8.3 added to the test matrix; CI now covers every version permitted by the updated composer constraint. |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[PHP Version] --> B{Permitted by composer.json?}
B -- "7.4 – 8.3 ✅" --> C{In CI matrix?}
B -- "8.4+ ❌ (excluded by <8.4)" --> X[Rejected at install]
C -- "7.4, 8.0, 8.1, 8.2, 8.3 ✅" --> D[Tested with prefer-lowest + prefer-stable]
D --> E[CI passes → safe to tag release]
%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
flowchart TD
A[PHP Version] --> B{Permitted by composer.json?}
B -- "7.4 – 8.3 ✅" --> C{In CI matrix?}
B -- "8.4+ ❌ (excluded by <8.4)" --> X[Rejected at install]
C -- "7.4, 8.0, 8.1, 8.2, 8.3 ✅" --> D[Tested with prefer-lowest + prefer-stable]
D --> E[CI passes → safe to tag release]
Reviews (2): Last reviewed commit: "fix: CRM-14077 narrow the lift to <8.4 —..." | Re-trigger Greptile
…range Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
The red 8.3 CI legs are expected and not a fault in this PR: composer resolution fails because the released 🤖 Generated with Claude Code |
Feature overview/Issue description
Part of the wings-api PHP 7.4 → 8.3 upgrade (CRM-14074, deps slice CRM-14077 group 5): this package's
<8.3PHP cap is one of the last composer-level blockers for running wings-api on 8.3.Summary of changes
composer.json: PHP constraint>=7.4 <8.3→>=7.4 <8.4Per the epic's design (D6): constraint-lift + 8.3 CI only — code changes happen only if this package's own 8.3 CI fails.
Blockers
🤖 Generated with Claude Code